-
Notifications
You must be signed in to change notification settings - Fork 10
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Flux Runtime Card and Page #55
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Really nice work in this, I think we need the docs, and I suspect we want to allow for non-flux-system namespace installations?
…using the helper in the Flux Runtime page
plugins/backstage-plugin-flux/src/components/FluxRuntimeCard/FluxRuntimeCard.test.tsx
Show resolved
Hide resolved
plugins/backstage-plugin-flux/src/components/FluxRuntimeCard/FluxRuntimeCard.test.tsx
Show resolved
Hide resolved
'kustomize.toolkit.fluxcd.io/name': 'default', | ||
'kustomize.toolkit.fluxcd.io/namespace': 'default', | ||
}, | ||
), |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should we include a case where perhaps one of the controllers has not been upgraded?
What would we show in that case?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Was just chatting with Simon about it and we'll switch to using the version in Namespace rather than this one
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@foot @bigkevmcd Just to sum up as we are discussing to show both the version in the controller and the one in the namespace. How do we want this to look like?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
So the base the user stories we want to cover:
- What version of flux do is installed
- What components of flux are installed
Additional questions we can answer on top (but this is kind of getting away from the dev user stories?)
- What namespace is flux installed into (99% flux-system)
- Is the flux install broken (mismatches in components versions)
For 1, we add the namespace column.
For 2, we could
- show every component's reported flux version by making each component a column (perhaps a bit info overload)
- show the versions in a tooltip on hover of the components list.
- something a bit more "clever" like, if component.flux-version !== ns.flux-version, show version in yellow with a warning next to the component.
Tooltip could work here?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
So the base the user stories we want to cover:
- What version of flux do is installed
- What components of flux are installed
Additional questions we can answer on top (but this is kind of getting away from the dev user stories?)
- What namespace is flux installed into (99% flux-system)
- Is the flux install broken (mismatches in components versions)
I agree that we need to not get away from dev user stories.
My original thinking
- It's useful broadly in terms of the Flux experience within Backstage
- If something isn't quite working, it might be useful to see the version of Flux on the cluster so that you know what's running to try and figure out why your Flux components might not be succeeding.
I was thinking about the case where a field had changed (or didn't exist) in a Flux resource that we show the status of, it would be useful to know what version is running in "that cluster" to understand why?
But it's easy to then think, well, you need to click through to Weave GitOps to view the resources in more detail, so maybe we don't need this at all, because you can go to Weave GitOps for this level of debugging information?
For 1, we add the namespace column.
For 2, we could
- show every component's reported flux version by making each component a column (perhaps a bit info overload)
This would serve two roles, it would provide the versions, but it would also highlight missing controllers across clusters.
As long as it scaled with the controllers (let's not add all controllers to the table unless we discover all of them).
- show the versions in a tooltip on hover of the components list.
- something a bit more "clever" like, if component.flux-version !== ns.flux-version, show version in yellow with a warning next to the component.
I like this idea, this feels really useful with low screen-overhead.
Tooltip could work here?
Yes, for versions of the controller, and I like the version colouration idea above too!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'd also be happy for this to be fixed up and landed, and we can always choose to make it better.
And add some instructions to the README that are basically this thing https://backstage.io/docs/getting-started/homepage/
|
@foot @bigkevmcd What Icon would you like to use for our |
Flux Icon would actually be kind of cool :) Do any other plugins have examples of bundling and distributing SVG icons etc? |
Which is exported by https://github.com/backstage/backstage/blob/master/plugins/github-pull-requests-board/src/components/icons/DraftPr/index.ts |
Something else to bear in mind, we may provide an Icon, users configure it, so...our docs should include an example including the icon, or worst case, a generic icon which is totally fine. I'd be happy if we included the Flux icon (subject to checking that we can! I think it should be ok, but something to bear in mind) |
…tage into flux-runtime
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🔥 !
Nice one
Co-authored-by: Simon <footless@gmail.com>
Closes #3437